build: Rename 'with-included-immodules' option
authorEmmanuele Bassi <ebassi@gnome.org>
Wed, 14 Feb 2018 13:43:19 +0000 (13:43 +0000)
committerEmmanuele Bassi <ebassi@gnome.org>
Wed, 14 Feb 2018 14:14:22 +0000 (14:14 +0000)
We can call it 'included-immodules', and simplify its logic by always
attempting to split the value, to avoid turning an array into a string
and then back into an array again.

meson_options.txt
modules/input/meson.build

index 42283abdd2c16e7a7fe9f735f8639c9ebd2a67a2..2f9dd80285886d8cde74b52efdf705435e977bfe 100644 (file)
@@ -26,8 +26,8 @@ option('enable-xinerama', type: 'combo', choices : ['yes', 'no', 'auto'], value
   description : 'Enable support for the Xinerama extension')
 option('dynamic-modules', type: 'boolean', value : true,
   description : 'Allow dynamic module loading')
-option('with-included-immodules', type: 'string', value : '',
-  description : 'Build the specified input methods into gtk (comma-separated list or "all" or "none")')
+option('included-immodules', type: 'string', value : 'none',
+  description : 'Build the specified input methods (comma-separated list, "all", or "none")')
 option('documentation', type: 'boolean', value: 'false',
   description : 'Build API reference and tools documentation')
 option('man-pages', type: 'boolean', value: 'false',
index 72171582dba8de7c9f8e27bbd07b6cb12be45358..e60dbcc59e072f2f4c7e6ea6b3eb92aa5688aa1a 100644 (file)
@@ -32,21 +32,15 @@ all_immodules = [
 all_immodules += backend_immodules
 
 # Allow building some or all immodules included
-included_immodules = get_option('with-included-immodules')
-if included_immodules == 'none'
-  included_immodules = ''
-elif included_immodules == 'all' or included_immodules == 'yes'
-  included_immodules = ','.join(all_immodules)
-endif
-
-if included_immodules == ''
-  have_included_immodules = false
+included_immodules = get_option('included-immodules').split(',')
+if included_immodules.contains('none')
   included_immodules = []
-else
-  have_included_immodules = true
-  included_immodules = included_immodules.split(',')
+elif included_immodules.contains('all')
+  included_immodules = all_immodules
 endif
 
+have_included_immodules = included_immodules.length() > 0
+
 foreach im: included_immodules
   if not all_immodules.contains(im)
     error('The specified input method "@0@" is not available (available methods: @1@)'.format(im, ', '.join(all_immodules)))
@@ -107,5 +101,3 @@ foreach m: method_defs
     endif
   endif
 endforeach
-
-# TODO: post-install gtk4-query-immodules run to create immodules.c